Skip to content

Add exclusion annotation for hosted deployments#141

Closed
jonesbr17 wants to merge 1 commit intoopenshift:masterfrom
jonesbr17:exclude-internal-openshift-hosted
Closed

Add exclusion annotation for hosted deployments#141
jonesbr17 wants to merge 1 commit intoopenshift:masterfrom
jonesbr17:exclude-internal-openshift-hosted

Conversation

@jonesbr17
Copy link
Copy Markdown
Contributor

Configures the CVO to exclude manifests in deployments with external control planes
See openshift/cluster-version-operator#252

Configures the CVO to exclude manifests in deployments with external control planes
See openshift/cluster-version-operator#252
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Nov 1, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign fedosin after the PR has been reviewed.
You can assign the PR to them by writing /assign @fedosin in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot requested review from JoelSpeed and lobziik November 1, 2021 12:58
@JoelSpeed
Copy link
Copy Markdown
Contributor

Just to be very explicitly clear in my head, this is intended to not deploy the operator on hypershift guest clusters because something else will be deploying the CCM within the hypershift host cluster?

@jonesbr17
Copy link
Copy Markdown
Contributor Author

After more discussion, it sounds like we won't be going down this path

@jonesbr17 jonesbr17 closed this Nov 1, 2021
@jonesbr17 jonesbr17 deleted the exclude-internal-openshift-hosted branch November 1, 2021 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants